Skip to content

Conversation

@kasiakoziol
Copy link
Collaborator

@kasiakoziol kasiakoziol commented Oct 31, 2025

Description

It updates Ansible role to ingestor (from standalone) for IngestorCluster.

Key Changes

  • Update Ansible role to ingestor (from standalone) for IngestorCluster

Testing and Verification

Automated tests.

Related Issues

Jira: https://splunk.atlassian.net/browse/CSPL-4002

PR Checklist

  • Code changes adhere to the project's coding standards.
  • Relevant unit and integration tests are included.
  • Documentation has been updated accordingly.
  • All tests pass locally.
  • The PR description follows the project's guidelines.

@kasiakoziol kasiakoziol changed the base branch from develop to CSPL-3551-ingestion-cr October 31, 2025 09:52
@coveralls
Copy link
Collaborator

coveralls commented Oct 31, 2025

Pull Request Test Coverage Report for Build 21821787625

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.007%) to 85.365%

Files with Coverage Reduction New Missed Lines %
pkg/splunk/enterprise/afwscheduler.go 1 92.75%
Totals Coverage Status
Change from base Build 19886830069: -0.007%
Covered Lines: 11269
Relevant Lines: 13201

💛 - Coveralls

@kasiakoziol kasiakoziol removed the request for review from gabrielm-splunk December 23, 2025 14:22
needs: unit-tests
env:
SPLUNK_ENTERPRISE_IMAGE: ${{ secrets.SPLUNK_ENTERPRISE_IMAGE }}
SPLUNK_ENTERPRISE_IMAGE: splunk/splunk:10.2
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Once this and the 3.0.0 release is verified with splunk 10.2, we can update the secret and remove the hard coding.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rlieberman-splunk Thanks. As initially planned, I will remove the commit once tests are green and will wait for the secret change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants